Skip to content

Conversation

blink-so[bot]
Copy link

@blink-so blink-so bot commented Sep 10, 2025

Overview

Updates README.md to accurately reflect the recent changes from the release automation PR, ensuring users have correct and up-to-date information.

📝 Key Updates

📦 Installation Section

  • GitHub Releases first - Now the primary installation method
  • All platforms covered - Download examples for Linux x64/ARM64, macOS Intel/Apple Silicon
  • Correct Go version - Updated from 1.21+ to 1.25+ to match CI workflows
  • Better organization - Releases first, build from source second

🚀 Quick Start Section

  • Installation options - Both releases and build from source
  • Assumes installation - Examples use jail instead of ./jail
  • Makefile mention - Shows make build as an option

🛠️ Development Section

  • Makefile primary - All available targets documented
  • Comprehensive tooling - build, build-all, test, test-coverage, clean, fmt, lint
  • Manual commands - Kept as secondary option for flexibility
  • Build script - Reference to ./scripts/build.sh for cross-platform builds

🔍 Accuracy Fixes

Issue Before After
Installation Build from source only GitHub Releases primary
Go Version 1.21+ 1.25+ (matches CI)
Build Commands Direct Go only Makefile primary
Examples ./jail jail (assumes installed)
Platforms Generic instructions Specific download examples

🎯 Benefits

Better User Experience

  • Easier installation - Download and extract vs building
  • Clear instructions - Step-by-step for each platform
  • Current information - Matches actual project state

Developer Friendly

  • Modern tooling - Makefile as primary development interface
  • All options covered - Both Makefile and manual commands
  • Consistent with CI - Same commands work locally and in CI

Accurate Documentation

  • No outdated info - Go version, build commands all current
  • Complete coverage - All 4 supported platforms documented
  • Proper flow - Installation → Usage → Development

🧪 Tested

  • ✅ All installation examples use correct URLs
  • ✅ Build commands work correctly (make build tested)
  • ✅ Go version requirement matches CI workflows
  • ✅ Platform examples cover all supported architectures

The README now provides accurate, up-to-date information that matches the current state of the project with automated releases and modern development tooling! 🎉

blink-so bot and others added 2 commits September 10, 2025 00:39
Updated README.md to be accurate with recent changes:

## Installation Section
- Added GitHub Releases as primary installation method
- Provided download examples for all 4 supported platforms
- Updated Go version requirement from 1.21+ to 1.25+
- Reorganized with releases first, build from source second

## Quick Start Section
- Added installation options (releases vs build from source)
- Updated examples to use 'jail' instead of './jail' (assumes installed)
- Mentioned Makefile as build option

## Development Section
- Updated to use Makefile targets as primary approach
- Added all available make targets (build, build-all, test, etc.)
- Kept manual Go commands as secondary option
- Added reference to build script for cross-platform builds

## Accuracy Improvements
- Corrected Go version requirement to match CI workflows
- Added proper installation instructions for all platforms
- Made Makefile the recommended development approach
- Maintained backward compatibility with direct Go commands

The README now accurately reflects the current state of the project
with automated releases, Makefile tooling, and proper installation options.

Co-authored-by: f0ssel <[email protected]>
…README

Simplified README by removing detailed sections:

## Removed Installation Prerequisites
- Linux kernel version requirements
- iptables/pfctl dependencies
- Go version requirements
- sudo access notes

## Removed TLS Interception Section
- CA certificate storage details
- Environment variable explanations
- Certificate generation process
- Usage examples with --no-tls-intercept

## Kept Essential Information
- --no-tls-intercept flag in command-line options (without description)
- Core functionality and usage examples
- Installation and development instructions

Rationale: Users will discover missing dependencies when they try to
use the tool. TLS interception details are implementation specifics
that users don't need to understand upfront.

The README is now more focused and less intimidating for new users.

Co-authored-by: f0ssel <[email protected]>
@f0ssel f0ssel merged commit 950039a into main Sep 10, 2025
7 checks passed
@f0ssel f0ssel deleted the blink/update-readme branch September 16, 2025 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant